From: Jasper St. Pierre Date: Mon, 24 Jun 2013 19:03:29 +0000 (-0400) Subject: dateMenu: Update visibility syncing for new dateMenu layout X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~4020 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=929636ebd0b8f428fbedb771e4123dc04f7242a1;p=gnome-shell.git dateMenu: Update visibility syncing for new dateMenu layout The code here before was trying to play hierarchy tricks to figure out how to show / hide the events list, which broke when we rearranged how the date menu was laid out. Simplify the code here to not be so tricky, and update the CSS to match the new designs. https://bugzilla.gnome.org/show_bug.cgi?id=702849 --- diff --git a/data/theme/gnome-shell.css b/data/theme/gnome-shell.css index 02168465e..72e14773c 100644 --- a/data/theme/gnome-shell.css +++ b/data/theme/gnome-shell.css @@ -1154,11 +1154,6 @@ StScrollBar StButton#vhandle:active { /* Calendar popup */ -#calendarArea { - /* this is the total width of the popup */ - width: 720px; -} - .calendar-vertical-separator { -stipple-width: 1px; -stipple-color: #505050; diff --git a/js/ui/dateMenu.js b/js/ui/dateMenu.js index 08c644aa4..85a0aded1 100644 --- a/js/ui/dateMenu.js +++ b/js/ui/dateMenu.js @@ -153,15 +153,14 @@ const DateMenuButton = new Lang.Class({ this._openClocksItem.actor.visible = visible && (this._getClockApp() != null); this._separator.visible = visible; + this._eventList.actor.visible = visible; if (visible) { - let alignment = 0.25; - if (Clutter.get_default_text_direction() == Clutter.TextDirection.RTL) - alignment = 1.0 - alignment; - this.menu._arrowAlignment = alignment; - this._eventList.actor.get_parent().show(); + let alignment = 0.25; + if (Clutter.get_default_text_direction() == Clutter.TextDirection.RTL) + alignment = 1.0 - alignment; + this.menu._arrowAlignment = alignment; } else { - this.menu._arrowAlignment = 0.5; - this._eventList.actor.get_parent().hide(); + this.menu._arrowAlignment = 0.5; } },